Skip to content

TPC TimeSeries: propagate min-momentum, min-cluster, max-tgl to workflow#15618

Open
miranov25 wants to merge 1 commit into
AliceO2Group:devfrom
miranov25:dev
Open

TPC TimeSeries: propagate min-momentum, min-cluster, max-tgl to workflow#15618
miranov25 wants to merge 1 commit into
AliceO2Group:devfrom
miranov25:dev

Conversation

@miranov25

Copy link
Copy Markdown
Contributor

Add configurable environment variables for o2-tpc-time-series-workflow:
TPCTIMESERIES_MIN_MOMENTUM (default: 0.2)
TPCTIMESERIES_MIN_CLUSTER (default: 80)
TPCTIMESERIES_MAX_TGL (default: 1.4)

Defaults match TPCTimeSeriesSpec.cxx hardcoded values. Previously these cuts were not configurable from the workflow script.

Add configurable environment variables for o2-tpc-time-series-workflow:
  TPCTIMESERIES_MIN_MOMENTUM (default: 0.2)
  TPCTIMESERIES_MIN_CLUSTER  (default: 80)
  TPCTIMESERIES_MAX_TGL      (default: 1.4)

Defaults match TPCTimeSeriesSpec.cxx hardcoded values.
Previously these cuts were not configurable from the workflow script.
@miranov25
miranov25 requested a review from a team as a code owner July 21, 2026 13:28
@miranov25

Copy link
Copy Markdown
Contributor Author

Hello @shahor02, @marslandALICE, @matthias-kleiner, and @catalinristea,

Here is the pull request needed for O2 to set parameters for the low-gain calibration.

We need it as a technical fix and will keep the old default values.

@miranov25

Copy link
Copy Markdown
Contributor Author

Analyzing the output of the latest test production – changing dEddxRation cut – we found another problem in the multiplicity selection.

We introduced multiplicity binning in the time series, but by mistake applied the same multiplicity cut not only for the time series binning, but also for the track selection.

The default value of the cut, 50000, does not allow us to perform studies at extreme occupancies.

To address this, we will need to modify the source code.
I will update the pull request.
Should we first make the value parametrizable from the jdl, or is it better to directly fix the source code?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant